Skip to content

Time - Alicia#34

Open
aecombs wants to merge 2 commits intoAda-C13:masterfrom
aecombs:master
Open

Time - Alicia#34
aecombs wants to merge 2 commits intoAda-C13:masterfrom
aecombs:master

Conversation

@aecombs
Copy link

@aecombs aecombs commented Mar 30, 2020

No description provided.

@tildeee
Copy link

tildeee commented Apr 3, 2020

Hi Alicia! Great work on this assignment!

Your answers look great! I only have one small comment: In your answers about GET Requests, for this assignment, I would expect that you needed to include the Slack token as a query parameter.

Your review looks great, too. It looks like you didn't get to wave 3/testing wave 3. That's okay, but it's important to me that you have a general understanding of how testing APIs is. Let me know if you have any questions on that material and we'll make time for it!

Other than that, things look good, well done!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants